Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: support for data stores #192

Open
wants to merge 42 commits into
base: master
Choose a base branch
from
Open

feat: support for data stores #192

wants to merge 42 commits into from

Conversation

xavidop
Copy link
Owner

@xavidop xavidop commented Dec 29, 2023

Summary by CodeRabbit

  • New Features

    • Introduced cxcli datastore command for managing datastores.
    • Added cxcli datastore search command for searching within datastores.
    • Implemented deletion functionality for datastores using cxcli.
  • Documentation

    • Added documentation for cxcli datastore and cxcli datastore search commands.
    • Introduced new sections for "Data Stores" with Introduction and Search pages.
    • Updated documentation to include new datastore commands and operations.
  • Development Environment

    • Added launch configurations for datastore commands in the development environment.
  • General Updates

    • Updated copyright year to 2024.

Summary by CodeRabbit

  • New Features

    • Introduced cxcli datastore command for managing datastore operations.
    • Added search functionality within datastore commands.
  • Bug Fixes

    • Updated import paths for compatibility with new package versions.
  • Documentation

    • Added new documentation for cxcli datastore commands and datastore operations.
    • Introduced documentation on creating and managing webhooks in Dialogflow CX.
  • Chores

    • Updated copyright year from 2023 to 2024.
    • Added .coderabbit.yaml for review configurations.
    • Updated GitHub workflow to use the latest version of anchore/sbom-action/download-syft.

@xavidop xavidop changed the title feat: support to data stores feat: support for data stores Jan 30, 2024
Copy link

stale bot commented Feb 13, 2024

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the wontfix This will not be worked on label Feb 13, 2024
@stale stale bot removed the wontfix This will not be worked on label Feb 13, 2024
Copy link

stale bot commented Mar 1, 2024

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the wontfix This will not be worked on label Mar 1, 2024
@stale stale bot closed this Mar 8, 2024
@xavidop xavidop reopened this Mar 8, 2024
@xavidop xavidop removed the wontfix This will not be worked on label Mar 8, 2024
@mergify mergify bot removed the conflict label May 16, 2024
@xavidop
Copy link
Owner Author

xavidop commented May 16, 2024

@coderabbitai review

Copy link

coderabbitai bot commented May 16, 2024

Actions Performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 0

Review Details

Configuration used: .coderabbit.yaml
Review profile: CHILL

Commits Files that changed from the base of the PR and between 74805ec and 4fd0b0c.
Files ignored due to path filters (2)
  • go.mod is excluded by !**/*.mod, !**/*.mod
  • go.sum is excluded by !**/*.sum, !**/*.sum
Files selected for processing (1)
  • .coderabbit.yaml (1 hunks)
Files skipped from review due to trivial changes (1)
  • .coderabbit.yaml

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 0

Review Details

Configuration used: .coderabbit.yaml
Review profile: CHILL

Commits Files that changed from the base of the PR and between 4fd0b0c and 7473634.
Files selected for processing (1)
  • .coderabbit.yaml (1 hunks)
Files skipped from review as they are similar to previous changes (1)
  • .coderabbit.yaml

Copy link

stale bot commented May 30, 2024

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the wontfix This will not be worked on label May 30, 2024
@xavidop xavidop removed the wontfix This will not be worked on label May 30, 2024
dependabot bot and others added 6 commits June 5, 2024 10:19
Bumps [golang.org/x/term](https://github.com/golang/term) from 0.20.0 to 0.21.0.
- [Commits](golang/term@v0.20.0...v0.21.0)

---
updated-dependencies:
- dependency-name: golang.org/x/term
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [google.golang.org/api](https://github.com/googleapis/google-api-go-client) from 0.182.0 to 0.183.0.
- [Release notes](https://github.com/googleapis/google-api-go-client/releases)
- [Changelog](https://github.com/googleapis/google-api-go-client/blob/main/CHANGES.md)
- [Commits](googleapis/google-api-go-client@v0.182.0...v0.183.0)

---
updated-dependencies:
- dependency-name: google.golang.org/api
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Xavier Portilla Edo <xavi_tb@hotmail.com>
Copy link
Contributor

mergify bot commented Jun 5, 2024

👋 xavidop your PR is conflicting and needs to be updated to be merged

@mergify mergify bot added the conflict label Jun 5, 2024
Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 0

Review details

Configuration used: .coderabbit.yaml
Review profile: CHILL

Commits

Files that changed from the base of the PR and between 7473634 and f4beab0.

Files ignored due to path filters (2)
  • go.mod is excluded by !**/*.mod, !**/*.mod
  • go.sum is excluded by !**/*.sum, !**/*.sum
Files selected for processing (2)
  • .github/workflows/release_build.yml (2 hunks)
  • .goreleaser.yml (2 hunks)
Files skipped from review due to trivial changes (1)
  • .goreleaser.yml
Additional comments not posted (1)
.github/workflows/release_build.yml (1)

33-33: Updated anchore/sbom-action/download-syft to v0.16.0 aligns with the PR objectives.

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 0

Review details

Configuration used: .coderabbit.yaml
Review profile: CHILL

Commits

Files that changed from the base of the PR and between f4beab0 and 878d4f0.

Files ignored due to path filters (2)
  • go.mod is excluded by !**/*.mod, !**/*.mod
  • go.sum is excluded by !**/*.sum, !**/*.sum
Files selected for processing (4)
  • internal/utils/version.go (1 hunks)
  • pkg/profileconversation/execute.go (2 hunks)
  • pkg/profilenlu/execute.go (2 hunks)
  • pkg/vertexai/palmclient.go (1 hunks)
Files skipped from review due to trivial changes (1)
  • internal/utils/version.go
Additional comments not posted (6)
pkg/vertexai/palmclient.go (2)

6-17: The refactoring to use the palm package in CreatePalmClient is well-implemented and maintains clean and conditional initialization based on credentials.


22-22: The implementation of GenerateText using the palm client is correct and effectively handles the text generation process.

pkg/profilenlu/execute.go (2)

12-12: Integration of the palm package in ExecuteSuite is correctly implemented, enhancing the function's capability to handle text generation for tests.


139-139: The modifications in getResponse to utilize the palm client for text generation are well-executed, maintaining robust error handling and clear logging.

pkg/profileconversation/execute.go (2)

14-14: Integration of the palm package in ExecuteSuite is correctly implemented, enhancing the function's capability to handle text generation for conversations.


368-368: The modifications in getResponse to utilize the palm client for text generation are well-executed, maintaining robust error handling and clear logging.

dependabot bot and others added 6 commits June 5, 2024 10:53
Bumps [golang.org/x/term](https://github.com/golang/term) from 0.20.0 to 0.21.0.
- [Commits](golang/term@v0.20.0...v0.21.0)

---
updated-dependencies:
- dependency-name: golang.org/x/term
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [google.golang.org/api](https://github.com/googleapis/google-api-go-client) from 0.182.0 to 0.183.0.
- [Release notes](https://github.com/googleapis/google-api-go-client/releases)
- [Changelog](https://github.com/googleapis/google-api-go-client/blob/main/CHANGES.md)
- [Commits](googleapis/google-api-go-client@v0.182.0...v0.183.0)

---
updated-dependencies:
- dependency-name: google.golang.org/api
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Xavier Portilla Edo <xavi_tb@hotmail.com>
Copy link

stale bot commented Jun 19, 2024

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the wontfix This will not be worked on label Jun 19, 2024
@xavidop xavidop removed the wontfix This will not be worked on label Jun 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant